Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add placeholder pseudo-element #548

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dzuk-mutant
Copy link

https://developer.mozilla.org/en-US/docs/Web/CSS/::placeholder

While the MDN article mentions there are accessibility concerns in using this, as long as you understand the constraints and work with/counteract default browser styling it can be a useful part of designing inputs IMO.

If this would be an obstacle to including it in elm-css, perhaps the docs can mention that it's something that designers need to be aware of these sorts of issues if they want to use it.

@dzuk-mutant dzuk-mutant changed the title Placeholder pseudo-element Add placeholder pseudo-element Nov 16, 2021
@dzuk-mutant
Copy link
Author

dzuk-mutant commented Nov 16, 2021

(In case this isn't fixed by the time someone who has control of repo sees it, GitHub seems to be having a bug with dummy email addresses right now, so the commits in this PR have been tagged as being made by a user who is not me but happened to use the same dummy email in the past. This is not intentional, I do not have control of that GitHub account 😓)

@dzuk-mutant dzuk-mutant changed the title Add placeholder pseudo-element Add placeholder pseudo-element Nov 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants